Skip to content

refactor wasm process code#505

Merged
Guest0x0 merged 5 commits into
mainfrom
cleanup-wasm-process
Jul 20, 2026
Merged

refactor wasm process code#505
Guest0x0 merged 5 commits into
mainfrom
cleanup-wasm-process

Conversation

@Guest0x0

@Guest0x0 Guest0x0 commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator
  • refactor code for process handling in @event_loop, align native FFI shape with Wasm and share the main MoonBit logic part
  • use the full native test suite for Wasm process testing

Code in @process itself is not touched, because some runtime refactor may be necessary.

@coveralls

coveralls commented Jul 16, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 887

Coverage decreased (-0.06%) to 76.885%

Details

  • Coverage decreased (-0.06%) from the base build.
  • Patch coverage: 9 uncovered changes across 3 files (17 of 26 lines covered, 65.38%).
  • 2 coverage regressions across 1 file.

Uncovered Changes

File Changed Covered %
src/internal/event_loop/event_loop_unix.mbt 4 0 0.0%
src/internal/event_loop/process.mbt 5 2 40.0%
src/internal/event_loop/process_unix.mbt 6 4 66.67%
Total (7 files) 26 17 65.38%

Coverage Regressions

2 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
src/http/client.mbt 2 79.63%

Coverage Stats

Coverage Status
Relevant Lines: 4867
Covered Lines: 3742
Line Coverage: 76.89%
Coverage Strength: 11600.45 hits per line

💛 - Coveralls

@Guest0x0
Guest0x0 force-pushed the cleanup-wasm-process branch 2 times, most recently from 5a0289d to d830859 Compare July 20, 2026 06:33
@Guest0x0
Guest0x0 force-pushed the cleanup-wasm-process branch from d830859 to 56ce36b Compare July 20, 2026 06:38
@Guest0x0
Guest0x0 marked this pull request as ready for review July 20, 2026 08:44
@Guest0x0
Guest0x0 merged commit 75af33e into main Jul 20, 2026
34 of 36 checks passed
@Guest0x0
Guest0x0 deleted the cleanup-wasm-process branch July 20, 2026 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants